-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrated bindata for public and templates #74
Conversation
LGTM |
Does public/ already support override via custom/public? |
I think It become a mess when two (or more) Pull Requests update different files (eg : app.ini for the first commit and add new files for the second) because the |
I tend to agree with @0xBAADF00D but would rather avoid blocking this change due to that issue. Rather, I suggest a new issue is filed for the bindata.go threatment. |
I'm currently thinking about rebuilding the bindata interface that handles static directories instead of embedding the data, switched via a build tag. |
Current coverage is 2.18% (diff: 0.00%)@@ master #74 diff @@
========================================
Files 31 31
Lines 7508 7508
Methods 0 0
Messages 0 0
Branches 0 0
========================================
Hits 164 164
Misses 7327 7327
Partials 17 17
|
@tboerger I don't know if it helps you, but I like fileb0x because it prevents the need of two different tools: bindata and bindata-assetfs. Also, bindata-assetfs has this bug on Windows. |
@andreynering +1 on not using go-bindata-assets WRT elazarl/go-bindata-assetfs#34 (comment)
|
fileb0x won't work because we rely on tags with the new bindata. |
Does Rice? https://github.com/GeertJohan/go.rice |
I will avaluate it with go.rice, looks like it also supports loading from a directory natively. |
Like this? UnnoTed/fileb0x#3 Just trying to help, I'm fine with any option 👍 |
@andreynering currently we want to embed the data optionally via a build tag, |
You're right, this would not be possible using fileb0x and build tags. A workaroud would be having two fileb0x files ( But if rice supports build tags, it would be way simpler. |
Fixes #30
Things left to do: